Skip to content

chore: lint fixes part 10#262

Open
pawbana wants to merge 1 commit intopb/coder-lint-fix9from
pb/coder-lint-fix10
Open

chore: lint fixes part 10#262
pawbana wants to merge 1 commit intopb/coder-lint-fix9from
pb/coder-lint-fix10

Conversation

@pawbana
Copy link
Copy Markdown
Contributor

@pawbana pawbana commented Apr 10, 2026

  • nilnil (4 → 0): added nolint:nilnil to intentional nil, nil returns in test mocks (MockProvider.CreateInterceptor, mockServerProxier.CallTool) and lastUserPrompt (no user prompt found is not an error)
  • gocritic FullResponseWriter (1 → 0): added Hijack() method to statusCapturingWriter delegating to underlying http.Hijacker
  • revive defer-in-loop (1 → 0): replaced defer shutdownCancel() with explicit shutdownCancel() call after events.Shutdown
  • staticcheck SA1026 (1 → 0): replaced responses.Error (which embeds *http.Request with unmarshalable GetBody func) with a plain struct matching the same JSON shape

All lint errors are now resolved — 0 remaining.

Copy link
Copy Markdown
Contributor Author

pawbana commented Apr 10, 2026

@pawbana pawbana marked this pull request as ready for review April 10, 2026 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant